Skip to content

keep current#880

Merged
henrydingliu merged 29 commits into
experimentalfrom
main
May 29, 2026
Merged

keep current#880
henrydingliu merged 29 commits into
experimentalfrom
main

Conversation

@henrydingliu
Copy link
Copy Markdown
Collaborator

@henrydingliu henrydingliu commented May 29, 2026

Summary of Changes

Related GitHub Issue(s)

Additional Context for Reviewers

  • I passed tests locally for both code (uv run pytest) and documentation changes (uv run jb build docs --builder=custom --custom-builder=doctest)

Note

Medium Risk
Datetime handling and options API changes affect core triangle valuation paths across the package; removing deprecated helpers is a breaking change for callers still using them.

Overview
Refactors package configuration so datetime precision follows Pandas via module-level __dt64_dtype__ / __dt64_unit__ instead of configurable options.DT64_*, and reworks Options as an instance with validated get/set/reset (including per-option reset). Removes the deprecated top-level array_backend() and auto_sparse() helpers in favor of chainladder.options.

Documentation and tests expand substantially: doctest Examples on Development, BootstrapODPSample, and Mack correlation classes; a new docs/README.md contributor guide; Friedland chapters 6–7 (RST) plus chapter 8 notebook wired into _toc.yml; and richer options unit tests plus a shared _sample_fixture in conftest.py.

Minor housekeeping: trailing newline on CODEOWNERS, pyproject.toml docs deps formatting, and library/api autosummary stub fix for Development.

Reviewed by Cursor Bugbot for commit 449b5c1. Bugbot is set up for automated code reviews on this repo. Configure here.

kennethshsu and others added 29 commits May 22, 2026 14:24
Adds Sphinx doctest Examples sections to DevelopmentCorrelation and ValuationCorrelation. Each example opens with the Mack chain-ladder assumption being tested, prints the full decision signal (statistic, confidence band, and boolean) rather than a single boolean, and ties the result back to the chain-ladder workflow.

Refs #704
Adds Sphinx doctest Examples section to the BootstrapODPSample class showing basic fit (resampled_triangles_.shape and scale_), downstream stochastic IBNR via Chainladder, and the effect of drop_high on scale_. Uses random_state=42 and n_sims=100 for deterministic, fast output.

Refs #704
Rewords the paragraph introducing the drop_high=True example to describe it as a leave-one-out sensitivity check on the column maxima rather than outlier removal, since drop_high mechanically removes the column max without any outlier test.

Addresses review feedback on #836.
Per @henrydingliu review on #844: opening paragraph now scoped to the calendar-effect concept; the per-diagonal vs whole-triangle distinction is introduced as a transition between the two testcode blocks.

Refs #704
- Drop the jb build internals + duplicate warnings note (kennethshsu)
- Rename 'tutorial notebooks' to 'onboarding and Quickstart notebooks' (kennethshsu)
- Drop the 'What to edit for which part of the site' table and 'Known issues' section as redundant with the Source files table and #841 (kennethshsu)
docs: add README documenting docs build sources and outputs
Improved docstring and added examples for Development class
[REFACTOR]: Remove repetitive code. Remove dead Python 3.8 code. Add …
…option getters and setters. Deprecate cl.array_backend() and cl.auto_sparse()
first batch of deliverables for friedland reconstruction
[REFACTOR]: Continue work on cl.__init__.py refactor
@henrydingliu henrydingliu requested a review from jbogaardt as a code owner May 29, 2026 21:33
@henrydingliu henrydingliu merged commit 4ce0141 into experimental May 29, 2026
43 checks passed
@codecov
Copy link
Copy Markdown

codecov Bot commented May 29, 2026

Codecov Report

❌ Patch coverage is 97.05882% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 87.04%. Comparing base (e76d85d) to head (449b5c1).
⚠️ Report is 157 commits behind head on experimental.

Files with missing lines Patch % Lines
chainladder/__init__.py 96.42% 1 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           experimental     #880      +/-   ##
================================================
+ Coverage         86.94%   87.04%   +0.09%     
================================================
  Files                86       86              
  Lines              4994     4986       -8     
  Branches            644      646       +2     
================================================
- Hits               4342     4340       -2     
+ Misses              462      456       -6     
  Partials            190      190              
Flag Coverage Δ
unittests 87.04% <97.05%> (+0.09%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants